Did this end up being a problem? I thought checks_run.sh was copied from flux-core and I'm not seeing a duplicate coverage.rc but I'm probably missing something that should be obvious, sorry. In any event, we should try to keep changes synced if you're going to continue to use the script from flux-core, so maybe we need a change there too?

No, not a problem. I opened this PR mostly as an experiment to see if removing the duplicate coverage.rc that I had in flux-accounting's checks_run.sh (I think due to my my negligence in #644) would actually trigger the Python bindings from actually getting picked up by Codecov, but I'm still not seeing coverage there. 🤷

I wonder if this error is the problem:

  No source for code: '/usr/src/usr/src/src/bindings/python/fluxacct/accounting/__init__.py'.

I'm not sure why the extra /usr/src, but maybe you can see if you add ignore_errors = True to the coverage.rc if you can get past that?

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.11%. Comparing base (ef310b5) to head (b596c9b).

@@           Coverage Diff           @@
##           master     #682   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files          26       26           
  Lines        2185     2185           
=======================================
  Hits         1838     1838           
  Misses        347      347